-
Notifications
You must be signed in to change notification settings - Fork 137
Provide client in activity context #740
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
962135c to
3164f69
Compare
|
This needs a bit more work -- as things stand it's attempting to pickle the client in the context in tests of process-based executor pools (which fails when it tries to pickle the |
abf45cb to
4577018
Compare
4577018 to
a8c177e
Compare
|
I'd be ok for now raising an exception for if the |
bd3b25e to
2f04c55
Compare
2f04c55 to
3c2c1bc
Compare
|
Rebased and ready for review again. Makes the worker's connected As discussed above, the client is only available in |
3c2c1bc to
08e0223
Compare
fc616a6 to
7ef37f6
Compare
* Expose client as activity.client()
Fixes #721